Merging this PR will not alter performance

✅ 9 untouched benchmarks

_{Comparing FarhanAliRaza:remove-json5-dep (24b214c) with main (d6db9c9)}

Greptile Summary

This PR removes the json5 npm dependency and replaces it with native JSON.parse plus a regex-based fallback that rewrites Python's non-standard bare float tokens (Infinity, -Infinity, NaN) before retrying. The happy path pays no extra cost; only payloads that contain specials go through the rewrite step.

• NaN → null (not NaN) is an intentional behavior change: state values of float('nan') now render as empty strings in the UI rather than "NaN". The test reflects this, but downstream users who relied on the NaN display will see a silent difference with no deprecation notice.

Confidence Score: 5/5

Safe to merge; all findings are P2 style/documentation suggestions with no runtime correctness issues.

The regex implementation correctly handles Python's non-finite float tokens and the string-literal-first alternation prevents false positives inside quoted values. Removing json5 is a clean, well-scoped change. Both open findings are P2: one asks for word-boundary guards that are not required for real payloads, and one notes the intentional NaN→null behavior change lacks a deprecation/changelog entry.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Socket.io packet received (str)"] --> B["JSON.parse(str)"]
    B -->|success| C["Return parsed object"]
    B -->|throws| D["rewriteBareNonFiniteFloats(str)"]
    D --> E["Regex: match string literals first\n(pass through unchanged)"]
    E --> F["Replace bare Infinity → 1e999\n-Infinity → -1e999\nNaN → null"]
    F --> G["JSON.parse(rewritten str)"]
    G -->|success| H["Return parsed object\n(NaN becomes null, ±Inf preserved)"]
    G -->|throws| I["Return false\n(malformed payload)"]

_{Reviews (1): Last reviewed commit: "refactor: drop json5 dep, handle non-fin..." | Re-trigger Greptile}